Skip to content

Make minValues and maxValues the same length as minMaxChannels#13845

Merged
sopel39 merged 1 commit intotrinodb:masterfrom
tangjiangling:make-array-length-same-as-minMaxChannels
Aug 25, 2022
Merged

Make minValues and maxValues the same length as minMaxChannels#13845
sopel39 merged 1 commit intotrinodb:masterfrom
tangjiangling:make-array-length-same-as-minMaxChannels

Conversation

@tangjiangling
Copy link
Copy Markdown
Member

Previously, the length of minValues and maxValues was the same as
channels, which may exist the case that some elements in minValues
and maxValues are null, so this commit keeps the length of minValues
and maxValues the same as minMaxChannels, avoiding the case that
some array elements may be null.

Description

Is this change a fix, improvement, new feature, refactoring, or other?

Improvement

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

Core query engine.

How would you describe this change to a non-technical end user or system administrator?

N/A.

Documentation

(x) No documentation is needed.

Release notes

(x) No release notes entries required.

Previously, the length of `minValues` and `maxValues` was the same as
`channels`, which may exist the case that some elements in `minValues`
and `maxValues` are null, so this commit keeps the length of `minValues`
and `maxValues` the same as `minMaxChannels`, avoiding the case that
some array elements may be null.
@cla-bot cla-bot bot added the cla-signed label Aug 25, 2022
@tangjiangling tangjiangling requested review from findepi, losipiuk and raunaqmorarka and removed request for findepi and losipiuk August 25, 2022 07:35
@tangjiangling tangjiangling added the no-release-notes This pull request does not require release notes entry label Aug 25, 2022
@tangjiangling tangjiangling requested a review from losipiuk August 25, 2022 07:36
@findepi findepi requested review from sopel39 and removed request for findepi August 25, 2022 07:40
@sopel39 sopel39 merged commit e59e948 into trinodb:master Aug 25, 2022
@tangjiangling tangjiangling deleted the make-array-length-same-as-minMaxChannels branch August 25, 2022 13:59
@github-actions github-actions bot added this to the 394 milestone Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed no-release-notes This pull request does not require release notes entry

Development

Successfully merging this pull request may close these issues.

2 participants